Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New add euslisp test #799

Closed
wants to merge 4 commits into from
Closed

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Aug 26, 2015

I moved euslisp test program for rtm-ros-robot-interface.l from hrpsys_ros_bridge_tutorials:
start-jsk/rtmros_tutorials#369
#783 (comment)

Currently datalogger and sequenceplayer are checked.
I'll add unstable rtc tests in another PR.

…lerobot.yaml, hrpsys_ros_bridge/euslisp/samplerobot-interface.l] Move samplerobot euslisp tests and setting files from hrpsys_ros_bridge_tutorials.
@@ -20,6 +20,7 @@ env:
matrix:
- ROS_DISTRO=hydro USE_DEB=true
- ROS_DISTRO=hydro USE_DEB=false NOT_TEST_INSTALL=true
- ROS_DISTRO=hydro USE_DEB=false NOT_TEST_INSTALL=true EXTRA_DEB="ros-hydro-roseus ros-hydro-euscollada ros-hydro-pr2eus" CATKIN_PARALLEL_JOBS='-p1'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Frequently ROS_DISTRO=hydro USE_DEB=false NOT_TEST_INSTALL=true EXTRA_DEB="ros-hydro-roseus ros-hydro-euscollada ros-hydro-pr2eus" fails because of compile error because of kill and memory shortage.

When I add CATKIN_PARALLEL_JOBS='-p1', travis seems to be OK.

@k-okada
Copy link
Member

k-okada commented Aug 26, 2015

I'm working on this to keep original log, please wait for a moment

@snozawa
Copy link
Contributor Author

snozawa commented Aug 26, 2015

What kinds of log are you working on?

@snozawa
Copy link
Contributor Author

snozawa commented Aug 26, 2015

Sorry, I see, thats git log:
#800

@k-okada
Copy link
Member

k-okada commented Aug 26, 2015

#800 is not perfect yet

@k-okada
Copy link
Member

k-okada commented Aug 26, 2015

done, please add your changes on #800

@snozawa
Copy link
Contributor Author

snozawa commented Aug 26, 2015

こちら
k-okada#7
にPRしてcommit分だけマージされたら、
#800
も更新されるんでしたっけ?

@k-okada
Copy link
Member

k-okada commented Aug 26, 2015

はい.それでもいいですし,
hub remote add -p k-okada
git checkout nozawa_branch
git merge k-okada hogehoge
とかしてもいいとおもいます.

とりあえず頂いた分を#800に追加しました.

◉ Kei Okada

On Wed, Aug 26, 2015 at 7:26 PM, Shunichi Nozawa [email protected]
wrote:

こちら
k-okada#7 k-okada#7
にPRしてcommit分だけマージされたら、
#800 #800
も更新されるんでしたっけ?


Reply to this email directly or view it on GitHub
#799 (comment)
.

@snozawa
Copy link
Contributor Author

snozawa commented Aug 26, 2015

ありがとうございます、travisが通るかも確認しておきます。

@snozawa
Copy link
Contributor Author

snozawa commented Aug 26, 2015

Solved via #800.

@snozawa snozawa closed this Aug 26, 2015
@snozawa snozawa deleted the new_add_euslisp_test branch August 31, 2015 01:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants